Skip to content

fix(lightspeed-mcp): refactored mcp config modal#3698

Open
ciiay wants to merge 4 commits into
redhat-developer:mainfrom
ciiay:rhidp-14843-mcp-config-modal-refactor
Open

fix(lightspeed-mcp): refactored mcp config modal#3698
ciiay wants to merge 4 commits into
redhat-developer:mainfrom
ciiay:rhidp-14843-mcp-config-modal-refactor

Conversation

@ciiay

@ciiay ciiay commented Jul 7, 2026

Copy link
Copy Markdown
Member

Hey, I just made a Pull Request!

For RHIDP-14843

Enhance the MCP server configure modal with PatternFly layout, server status and tools list, aligned enabled toggle behavior, personal token save/remove flows (including disconnecting state and warnings), auto-enable on successful token validation, and unit tests.

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

Test steps:

Test steps for Intelligent-assistant plugin

screen recording

rhidp_14834.mp4

Signed-off-by: Yi Cai <yicai@redhat.com>
@rhdh-gh-app

rhdh-gh-app Bot commented Jul 7, 2026

Copy link
Copy Markdown

Important

This PR includes changes that affect public-facing API. Please ensure you are adding/updating documentation for new features or behavior.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-lightspeed workspaces/lightspeed/plugins/lightspeed minor v2.9.1

ciiay added 2 commits July 7, 2026 10:12
Signed-off-by: Yi Cai <yicai@redhat.com>
Signed-off-by: Yi Cai <yicai@redhat.com>
@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 79.42584% with 43 lines in your changes missing coverage. Please review.
✅ Project coverage is 54.58%. Comparing base (904c783) to head (b8b000b).
⚠️ Report is 4 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3698      +/-   ##
==========================================
+ Coverage   54.31%   54.58%   +0.26%     
==========================================
  Files        2349     2350       +1     
  Lines       89701    89859     +158     
  Branches    25127    25198      +71     
==========================================
+ Hits        48722    49047     +325     
+ Misses      40685    40516     -169     
- Partials      294      296       +2     
Flag Coverage Δ *Carryforward flag
adoption-insights 83.70% <ø> (ø) Carriedforward from a0fe739
ai-integrations 68.35% <ø> (ø) Carriedforward from a0fe739
app-defaults 69.79% <ø> (ø) Carriedforward from a0fe739
augment 46.39% <ø> (ø) Carriedforward from a0fe739
boost 74.68% <ø> (ø) Carriedforward from a0fe739
bulk-import 72.46% <ø> (ø) Carriedforward from a0fe739
cost-management 14.10% <ø> (ø) Carriedforward from a0fe739
dcm 61.81% <ø> (ø) Carriedforward from a0fe739
extensions 61.53% <ø> (ø) Carriedforward from a0fe739
global-floating-action-button 71.18% <ø> (ø) Carriedforward from a0fe739
global-header 59.71% <ø> (ø) Carriedforward from a0fe739
homepage 50.23% <ø> (ø) Carriedforward from a0fe739
install-dynamic-plugins 56.77% <ø> (ø) Carriedforward from a0fe739
konflux 91.49% <ø> (ø) Carriedforward from a0fe739
lightspeed 72.60% <79.42%> (+3.78%) ⬆️
mcp-integrations 85.46% <ø> (ø) Carriedforward from a0fe739
orchestrator 39.89% <ø> (ø) Carriedforward from a0fe739
quickstart 65.63% <ø> (ø) Carriedforward from a0fe739
sandbox 79.56% <ø> (ø) Carriedforward from a0fe739
scorecard 82.67% <ø> (ø) Carriedforward from a0fe739
theme 61.26% <ø> (ø) Carriedforward from a0fe739
translations 7.25% <ø> (ø) Carriedforward from a0fe739
x2a 78.68% <ø> (ø) Carriedforward from a0fe739

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 904c783...b8b000b. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Yi Cai <yicai@redhat.com>
@sonarqubecloud

sonarqubecloud Bot commented Jul 7, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant